fix(web): restore dividers beside sidebar section counts - #26403

Approvability

Verdict: Approved at ea1c1e8

Macroscope's review found this PR approvable — Restores decorative dividers beside sidebar counts and workflow phase details. The change is visual-only, preserving interaction behavior, accessibility attributes, and theme colors.

^{You can add or adjust custom eligibility rules. Learn more.}

No actionable comments were generated in the recent review.

Recent review info

Run configuration

• Source mode: Auto
• Repository: pingdotgg/t3code
• Branch: main
• File: .coderabbit.config.ts
• Commit: 2a73827
• Layers: Repository TS config
• Review profile: CHILL
• Plan: Advanced
• Run ID: 204b8efa-0873-4c05-b14c-d261be8f7390

Commits

Reviewing files that changed from the base of the PR and between 2a73827 and ea1c1e8.

Files selected for processing: 1

• apps/web/src/components/ui/collapsible-section-header.tsx

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 9 remain after this review.

Merge Risk: Minimal · up to ea1c1

The focused visual fix is mergeable with no outstanding review risks.

Pre-merge checks: 4

Check name	Status	Explanation
Title check		The title names the fix (restoring dividers) and the area it touches (sidebar section counts), which matches the change. The raw summary shows the accessory-based hiding condition was removed so the h…
Description check		The description covers the problem (sidebar lost dividers during the redesign), the change (removes the condition hiding the divider when an accessory is present), and verification (validation command…
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.

Full details: Title check

Explanation

The title names the fix (restoring dividers) and the area it touches (sidebar section counts), which matches the change. The raw summary shows the accessory-based hiding condition was removed so the horizontal rule stays visible. The title follows the conventional commit format and is specific. The description notes the same component also affects workflow phase headers, which the title does not mention, but that is a secondary effect and does not make the title misleading.

Full details: Description check

Explanation

The description covers the problem (sidebar lost dividers during the redesign), the change (removes the condition hiding the divider when an accessory is present), and verification (validation commands listed, before/after screenshots embedded, no new tests explained for a CSS-only change). It does not follow the template's Scope and approval section: it links no triaged issue or maintainer-approved discussion and gives no justification for skipping one. The Problem and Change headings are present in content form but not as the template headings. The check still passes because the description is mostly complete and the missing approval link is a process gap rather than a content gap for this small fix.

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

_{Comment @coderabbitai help to get the list of available commands.}

Verified in a real dev environment

verified as-is at ea1c1e8bee48869240408af4dcafdde90c96c800. no source changes or pushes.

• reproduced on github/main (5ef56853ef1854da50bc1909c6c0a32700f2cb13) in a separate worktree after vp i. working, snoozed and settled headers all hid their divider beside the count.
• installed the exact pr head with vp i, stopped main's captured process group, and ran one background vp run dev --home-dir /home/user/t3code/.t3. actual ports: web 5733, server 13773.
• drove the real web client using the repo's playwright dependency and /usr/bin/google-chrome. added the real git project through the app, created threads and ran codex gpt-6.1-sol turns through the local proxy. no mocks, fixtures or demo data. copied only the isolated, app-created state after stopping main for matching before/after data.
• on the pr, all three dividers remain visible while expanding and collapsing shelves during a real provider turn. counts and chevrons remain aligned. checked light at 1440×1000 and dark at 1440×1000 and 1100×800.
• checked neighboring behavior: wake removed the snoozed shelf at zero; re-snooze restored count 1. settling changed settled 1→2; un-settling restored 1. the real codex tool call completed and its reply rendered in the client.
• inspected the captures and played the uploaded 10-second mp4 through to the end and sought within it. the inline gif contains the same recording. a fresh github read reports 24 successful and 6 skipped contexts, with none failing or pending; no new source checks were run because this verification made no code changes.

before, expanded shelves (main):

after, same threads and counts (pr; elapsed labels advance naturally):

short runtime transcript:

main: Working1 / Snoozed1 / Settled1 -> divider visibility: hidden
pr:   Working1 / Snoozed1 / Settled1 -> divider visibility: visible
web composer -> codex gpt-6.1-sol -> timeout 60 sleep 45
provider turn -> completed, error=null
client reply -> "real codex sidebar check passed."
wake -> Snoozed0 (shelf removed); re-snooze -> Snoozed1
settle -> Settled2; un-settle -> Settled1

Not verified

• the shared component's claude workflow phase header path. these runs used codex; no claude workflow was launched, so its phase counters, failed status and elapsed accessory are unverified.
• native electron, ios and android clients; macos/windows-specific rendering; relay/tunnel and multi-device connections. this pass exercised the linux web client against the running local server.